Skip to content

fix(plugin-import-export): hide invalid sortBy options#11676

Merged
paulpopus merged 3 commits intopayloadcms:mainfrom
akhrarovsaid:fix/importexport-invalid-sort
Apr 11, 2026
Merged

fix(plugin-import-export): hide invalid sortBy options#11676
paulpopus merged 3 commits intopayloadcms:mainfrom
akhrarovsaid:fix/importexport-invalid-sort

Conversation

@akhrarovsaid
Copy link
Copy Markdown
Contributor

@akhrarovsaid akhrarovsaid commented Mar 13, 2025

What?

This PR aims to prevent runtime 500-level errors when using plugin-import-export by omitting invalid options such as arrays and blocks from the SortyBy component. This PR also omits the sort when it is false-y.

Why?

To prevent errors and show valid sortable fields to sort by.

How?

Omitting the sort when it is false-y from the preview, and omitting invalid sortBy options.

Demo (After):

Pages---Payload.webm

@paulpopus paulpopus merged commit 9d6bf0b into payloadcms:main Apr 11, 2026
805 of 815 checks passed
@akhrarovsaid akhrarovsaid deleted the fix/importexport-invalid-sort branch April 11, 2026 03:10
Raghvendra2420 pushed a commit to Raghvendra2420/payload that referenced this pull request Apr 13, 2026
)

<!--

Thank you for the PR! Please go through the checklist below and make
sure you've completed all the steps.

Please review the
[CONTRIBUTING.md](https://github.com/payloadcms/payload/blob/main/CONTRIBUTING.md)
document in this repository if you haven't already.

The following items will ensure that your PR is handled as smoothly as
possible:

- PR Title must follow conventional commits format. For example, `feat:
my new feature`, `fix(plugin-seo): my fix`.
- Minimal description explained as if explained to someone not
immediately familiar with the code.
- Provide before/after screenshots or code diffs if applicable.
- Link any related issues/discussions from GitHub or Discord.
- Add review comments if necessary to explain to the reviewer the logic
behind a change

### What?

### Why?

### How?

Fixes #

-->
### What?
This PR aims to prevent runtime 500-level errors when using
`plugin-import-export` by omitting invalid options such as arrays and
blocks from the `SortyBy` component. This PR also omits the `sort` when
it is false-y.

### Why?
To prevent errors and show valid sortable fields to sort by.

### How?
Omitting the sort when it is false-y from the preview, and omitting
invalid `sortBy` options.

Demo (After):


[Pages---Payload.webm](https://github.com/user-attachments/assets/fca07160-8d66-44dc-a19f-0b32a93050f2)

---------

Co-authored-by: Paul Popus <paul@payloadcms.com>
@github-actions
Copy link
Copy Markdown
Contributor

🚀 This is included in version v3.83.0

milamer pushed a commit to milamer/payload that referenced this pull request Apr 20, 2026
)

<!--

Thank you for the PR! Please go through the checklist below and make
sure you've completed all the steps.

Please review the
[CONTRIBUTING.md](https://github.com/payloadcms/payload/blob/main/CONTRIBUTING.md)
document in this repository if you haven't already.

The following items will ensure that your PR is handled as smoothly as
possible:

- PR Title must follow conventional commits format. For example, `feat:
my new feature`, `fix(plugin-seo): my fix`.
- Minimal description explained as if explained to someone not
immediately familiar with the code.
- Provide before/after screenshots or code diffs if applicable.
- Link any related issues/discussions from GitHub or Discord.
- Add review comments if necessary to explain to the reviewer the logic
behind a change

### What?

### Why?

### How?

Fixes #

-->
### What?
This PR aims to prevent runtime 500-level errors when using
`plugin-import-export` by omitting invalid options such as arrays and
blocks from the `SortyBy` component. This PR also omits the `sort` when
it is false-y.

### Why?
To prevent errors and show valid sortable fields to sort by.

### How?
Omitting the sort when it is false-y from the preview, and omitting
invalid `sortBy` options.

Demo (After):


[Pages---Payload.webm](https://github.com/user-attachments/assets/fca07160-8d66-44dc-a19f-0b32a93050f2)

---------

Co-authored-by: Paul Popus <paul@payloadcms.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants